- 
                Notifications
    You must be signed in to change notification settings 
- Fork 664
          Update CHANGELOG.md for 0.52.0 release, add scripts/ instructions for ASF releases
          #1479
        
          New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
0.52.0 release0.52.0 release, update instructions for ASF releases
      bb80c78    to
    1f904b1      
    Compare
  
    0.52.0 release, update instructions for ASF releases0.52.0 release, add scripts/ instructions for ASF releases
      a36bef8    to
    e9e1e63      
    Compare
  
    Tweak release script
e9e1e63    to
    a4aa6e2      
    Compare
  
    | under the License. | ||
| --> | ||
|  | ||
| # sqlparser-rs 0.52.0 Changelog | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is made with an excellent script originally written by @andygrove for DataFusion ❤️
| --> | ||
|  | ||
|  | ||
| ## Process Overview | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is liberally based off (aka copy/paste) from https://github.com/apache/datafusion/blob/main/dev/release/README.md
| @@ -1,81 +0,0 @@ | |||
| <!--- | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed old version of release instructions
| version = "0.51.0" | ||
| authors = ["Andy Grove <[email protected]>"] | ||
| homepage = "https://github.com/sqlparser-rs/sqlparser-rs" | ||
| version = "0.52.0" | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated to reflect apache author (sorry @andygrove !)
| @@ -0,0 +1,59 @@ | |||
| #!/usr/bin/python | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
copied from datafusion without modification
| @@ -0,0 +1,152 @@ | |||
| #!/bin/bash | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tested this locally and it seems to work
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
| Thanks @iffyio -- Let's get this in and give making a release a try! | 
| 🚀 | 
Part of #1423
This is the first release that will be officially voted / approved by the Apache voting process and this PR
dev/release/README.mdDraft until:
Following the directions results in this:
You can actually check out a release candidate I made here: